Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Single threaded LSP concurrency model #3341

Closed
wants to merge 1 commit into from
Closed

Conversation

emcfarlane
Copy link
Contributor

Removes the locking structures from the LSP server. Requests are processed serially. This simplifies the implementation, but may need to be revisited later.

Copy link
Contributor

The latest Buf updates on your PR. Results from workflow Buf CI / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedSep 23, 2024, 9:51 PM

@emcfarlane
Copy link
Contributor Author

Closing. Chatted offline, concurrency primitives will be needed later.

@emcfarlane emcfarlane closed this Sep 23, 2024
@emcfarlane emcfarlane deleted the ed/lspNoLocks branch September 23, 2024 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant